Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

BugFix : Fixed documentation issue for azure_rm_loadbalancer_info #719

Merged
merged 2 commits into from
Mar 11, 2022
Merged

BugFix : Fixed documentation issue for azure_rm_loadbalancer_info #719

merged 2 commits into from
Mar 11, 2022

Conversation

aparna-patil
Copy link
Contributor

SUMMARY

Fixes #299

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

azure_rm_loadbalancer_info

ADDITIONAL INFORMATION

@Fred-sun
Copy link
Collaborator

Fred-sun commented Jan 6, 2022

@aparna-patil These files all have the same problem. Please agree to change（ delete two line). Thank you very much!

tests/sanity/ignore-2.10.txt:
tests/sanity/ignore-2.11.txt:
tests/sanity/ignore-2.12.txt:
tests/sanity/ignore-2.13.txt:
ERROR: tests/sanity/ignore-2.13.txt:310:1: A100: Ignoring 'parameter-type-not-in-doc' on 'plugins/modules/azure_rm_loadbalancer_info.py' is unnecessary
ERROR: tests/sanity/ignore-2.13.txt:311:1: A100: Ignoring 'parameter-list-no-elements' on 'plugins/modules/azure_rm_loadbalancer_info.py' is unnecessary

@Fred-sun Fred-sun added documentation-pr Improvements or additions to documentation medium_priority Medium priority work in In trying to solve, or in working with contributors labels Jan 6, 2022
@aparna-patil
Copy link
Contributor Author

@aparna-patil These files all have the same problem. Please agree to change（ delete two line). Thank you very much!

tests/sanity/ignore-2.10.txt:
tests/sanity/ignore-2.11.txt:
tests/sanity/ignore-2.12.txt:
tests/sanity/ignore-2.13.txt:
ERROR: tests/sanity/ignore-2.13.txt:310:1: A100: Ignoring 'parameter-type-not-in-doc' on 'plugins/modules/azure_rm_loadbalancer_info.py' is unnecessary
ERROR: tests/sanity/ignore-2.13.txt:311:1: A100: Ignoring 'parameter-list-no-elements' on 'plugins/modules/azure_rm_loadbalancer_info.py' is unnecessary

Hi @Fred-sun Done! updated all these files. Thank you very much!

@Fred-sun Fred-sun added ready_for_review The PR has been modified and can be reviewed and merged and removed work in In trying to solve, or in working with contributors labels Mar 9, 2022
@xuzhang3
Copy link
Collaborator

LGTM

@xuzhang3 xuzhang3 merged commit 8b6c6c9 into ansible-collections:dev Mar 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@xuzhang3 xuzhang3

Labels
documentation-pr Improvements or additions to documentation medium_priority Medium priority ready_for_review The PR has been modified and can be reviewed and merged
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Wrong return value documentation for azure.azcollection.azure_rm_loadbalancer_info
3 participants
@aparna-patil @Fred-sun @xuzhang3